Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/helper resolver #130

Merged
merged 3 commits into from
Apr 19, 2017
Merged

Conversation

swey
Copy link
Contributor

@swey swey commented Apr 18, 2017

Added helperResolver config option to override the default helper resolution.

This config option is implemented in the same way the partialResolver is implemented and was added for the same reasons.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 86.726% when pulling 50c20fc on swey:feature/helper-resolver into e344e14 on pcardune:master.

Copy link
Owner

@pcardune pcardune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for writing tests and updating the docs! Just a couple small changes and this will be good to merge.

CHANGELOG.md Outdated
## [1.4.1] - 2017-04-18

### Added
- Added `helperResolver` config option to override the default helper resolution
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just put this under the [Unreleased] section, and I'll update with a version number and date when I get around to making a release (which will hopefully be soon).

package.json Outdated
@@ -28,7 +28,7 @@
"type": "git",
"url": "git://github.com/pcardune/handlebars-loader.git"
},
"version": "1.4.0",
"version": "1.4.1",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

likewise ditch this change. I'll update when I make a release.

README.md Outdated
handlebarsLoader: {
helperResolver: function(helper, callback){
// should pass the helper's path on disk to
// to the callback if one was find for the given parameter.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is an extra "to" in here. Also replace "find" with "found".

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 86.726% when pulling f840bc5 on swey:feature/helper-resolver into e344e14 on pcardune:master.

@pcardune pcardune merged commit 44f2d0a into pcardune:master Apr 19, 2017
@swey swey deleted the feature/helper-resolver branch April 19, 2017 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants